-
Notifications
You must be signed in to change notification settings - Fork 38.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Per Volume Inode Accounting #35132
Per Volume Inode Accounting #35132
Conversation
Jenkins GKE smoke e2e failed for commit 3649f6c. Full PR test history. The magic incantation to run this job again is |
Jenkins GCI GKE smoke e2e failed for commit 3649f6c. Full PR test history. The magic incantation to run this job again is |
Jenkins GCE e2e failed for commit 3649f6c. Full PR test history. The magic incantation to run this job again is |
Jenkins Kubemark GCE e2e failed for commit 3649f6c. Full PR test history. The magic incantation to run this job again is |
Jenkins GCE Node e2e failed for commit 3649f6c. Full PR test history. The magic incantation to run this job again is |
Jenkins GCE etcd3 e2e failed for commit 3649f6c. Full PR test history. The magic incantation to run this job again is |
Jenkins GCI GCE e2e failed for commit 3649f6c. Full PR test history. The magic incantation to run this job again is |
Jenkins unit/integration failed for commit 3649f6c. Full PR test history. The magic incantation to run this job again is |
Jenkins verification failed for commit 3649f6c. Full PR test history. The magic incantation to run this job again is |
pkg/volume/metrics_du.go, line 76 at r1 (raw file):
s/runDu/runFind Comments from Reviewable |
3649f6c
to
8593d87
Compare
… uses these are included in the summary
8593d87
to
d494ef6
Compare
@dchen1107 this should be ready to merge once you review it. |
LGTM. Can you add a node e2e test which has a pod with an empty volume and create some files and check /summary if properly reporting the number of inodes of such volume? You can send that test in a separate pr. Thanks! |
The end to end test: #33955. |
Automatic merge from submit-queue |
Automatic merge from submit-queue Fix the crossbuild that #35132 broke @dashpole @dchen1107 @vishh A quick LGTM would be nice in order to not block any releases.
Collects volume inode stats using the same find command as cadvisor. The command is "find path -xdev -printf '.' | wc -c". The output is passed to the summary api, and will be consumed by the eviction manager.
This cannot be merged yet, as it depends on changes adding the InodesUsed field to the summary api, and the eviction manager consuming this. Expect tests to fail until this happens.
DEPENDS ON #35137
This change is